-
Notifications
You must be signed in to change notification settings - Fork 1
All Changes related to content editor and Mint Component branding #444
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
…h/program-tools into feature/content-editor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR appears to be a comprehensive refactoring focused on standardizing styling and theming across the Mint Components library. The main purpose is to implement a unified branding configuration system while improving code organization and consistency.
Key changes include:
- Implementation of a dynamic branding configuration system with CSS custom properties
- Consolidation of import statements and file structure reorganization
- Replacement of hardcoded style values with configurable CSS variables
- Updates to component styling to use the new theming system
Reviewed Changes
Copilot reviewed 274 out of 282 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
packages/stencilbook/src/stencil-storybook.tsx | Code formatting changes from spaces to tabs |
packages/mint-components/src/utils/utils.ts | Added utility function for capitalizing strings with debug logging |
packages/mint-components/src/utils/parseStates.ts | New utility for parsing state configuration objects |
packages/mint-components/src/templates/*.html | Updated border styles to use CSS custom properties |
packages/mint-components/src/tables/GenericTableView.tsx | Replaced hardcoded colors with CSS variables |
packages/mint-components/src/stories/PortalTemplates.stories.tsx | Reorganized imports and removed readme references |
packages/mint-components/src/saasquatch.d.ts | Added comprehensive branding configuration interface |
packages/mint-components/src/insertfont.ts | New utility for dynamic font insertion |
packages/mint-components/src/index.html | Added branding configuration script and structure |
packages/mint-components/src/global/styles.ts | Major refactoring to support dynamic branding with CSS custom properties |
packages/mint-components/src/global/mixins.ts | Updated color references to use new CSS variables |
packages/mint-components/src/global/global.ts | Added branding configuration initialization logic |
packages/mint-components/src/components/views/*.tsx | Updated styling to use new theming system |
packages/mint-components/src/components/tax-and-cash/**/*.tsx | Extensive updates to use new styling system and fix import paths |
Files not reviewed (2)
- packages/bedrock-components/package-lock.json: Language not supported
- packages/mint-components/package-lock.json: Language not supported
Comments suppressed due to low confidence (5)
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 274 out of 282 changed files in this pull request and generated no new comments.
Files not reviewed (2)
- packages/bedrock-components/package-lock.json: Language not supported
- packages/mint-components/package-lock.json: Language not supported
Comments suppressed due to low confidence (4)
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 274 out of 282 changed files in this pull request and generated no new comments.
Files not reviewed (2)
- packages/bedrock-components/package-lock.json: Language not supported
- packages/mint-components/package-lock.json: Language not supported
Comments suppressed due to low confidence (5)
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
…h Transparent class and transparent prop
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 275 out of 283 changed files in this pull request and generated no new comments.
Files not reviewed (2)
- packages/bedrock-components/package-lock.json: Language not supported
- packages/mint-components/package-lock.json: Language not supported
Comments suppressed due to low confidence (5)
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 275 out of 283 changed files in this pull request and generated no new comments.
Files not reviewed (2)
- packages/bedrock-components/package-lock.json: Language not supported
- packages/mint-components/package-lock.json: Language not supported
Comments suppressed due to low confidence (8)
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Description of the change
Type of change
Links
Checklists
Development
Paperwork
Code review